Skip to content

Clarify required and optional TRX report metadata - #56891

Merged
Evangelink merged 1 commit into
dotnet:mainfrom
Evangelink:dev/amauryleve/official-trx-documentation
Nov 13, 2026
Merged

Evangelink merged 1 commit into
dotnet:mainfrom
Evangelink:dev/amauryleve/official-trx-documentation

Conversation

@Evangelink

@Evangelink Evangelink commented Nov 13, 2026 •

Copy link
Copy Markdown
Member

Summary

Framework authors need to know which test-node data TRX requires and which properties only enrich the report. The existing guidance omits required identity data and overstates the fallback when a framework doesn't advertise TRX support.

  • Distinguish final-state and identity requirements from optional timing, failure details, output, categories, metadata, and attachments.
  • Document timing and name fallbacks, repeated-UID consistency, and property-container rules without changing reporter behavior.
  • Link the TRX reporting and framework guides to the requirements, preserving the existing TRX anchor and examples.

The guidance reflects the reviewed TestFx implementation and verified Microsoft.Testing.Platform 2.5.1 behavior. This PR changes documentation only.

Relates to microsoft/testfx#12436.


Internal previews

File Preview link
docs/core/testing/microsoft-testing-platform-architecture-capabilities.md Learn preview
docs/core/testing/microsoft-testing-platform-architecture-test-framework.md Learn preview
docs/core/testing/microsoft-testing-platform-test-reports.md Learn preview

Build report

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@Evangelink
Evangelink requested review from a team and BillWagner as code owners November 13, 2026 13:30
Copilot AI balanced review requested due to automatic review settings November 13, 2026 13:30
@dotnetrepoman dotnetrepoman Bot added this to the November 2026 milestone Nov 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

The precise MTP 2.5.1 and TRX runtime guarantees need framework-maintainer validation because access restrictions prevented verification against the relevant TestFx source and issues.

No open issues 🎉

What changed in this PR

Clarifies the TRX data contract for test-framework authors in the Microsoft.Testing.Platform documentation. The changes distinguish required result data from optional report enrichment without changing reporter behavior.

Changes:

  • Document required final-state and identity properties separately from optional metadata.
  • Explain timing and name fallbacks, repeated-UID consistency, property containers, and capability warnings.
  • Link the reporting and framework guides to the requirements while preserving existing examples and the TRX anchor.
File Description
docs/​core/​testing/​microsoft-testing-platform-test-reports.md Adds identity requirements and links to required and optional data guidance.
docs/​core/​testing/​microsoft-testing-platform-architecture-test-framework.md Directs framework authors to required identity data on final node updates.
docs/​core/​testing/​microsoft-testing-platform-architecture-capabilities.md Details TRX requirements, optional enrichment, fallbacks, and consistency rules.

Review effort: Balanced

@Evangelink
Evangelink enabled auto-merge (squash) November 13, 2026 13:43
@Evangelink
Evangelink merged commit dd99230 into dotnet:main Nov 13, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants